Issues

src/common/Knuth.java

Use "java.util.Random.nextInt()" instead.

L36

5min effort

Use "java.util.Random.nextInt()" instead.

L54

5min effort

src/common/Queue.java

Rename this generic name to match the regular expression '^[A-Z][0-9]?$'.

L26

10min effort

src/common/RandomGenerator.java

Save and re-use this "Random".

L15

5min effort

src/common/RandomSet.java

Override the "equals" method in this class.

L5

30min effort

src/common/StdIn.java

Do not forget to remove this deprecated code someday.

L587

10min effort

Do not forget to remove this deprecated code someday.

L601

10min effort

Do not forget to remove this deprecated code someday.

L612

10min effort

src/common/StdOut.java

Replace this use of System.out or System.err by a logger.

L68

10min effort

Replace charset name argument with StandardCharsets.UTF_8

L68

1min effort

Replace this use of System.out or System.err by a logger.

L70

10min effort

Replace this use of System.out or System.err by a logger.

L321

10min effort

src/custom/fattree/Address.java

Rename this field "_1" to match the regular expression '^[a-z][a-zA-Z0-9]*$'.

L4

2min effort

Rename this field "_2" to match the regular expression '^[a-z][a-zA-Z0-9]*$'.

L5

2min effort

Rename this field "_3" to match the regular expression '^[a-z][a-zA-Z0-9]*$'.

L6

2min effort

Rename this field "_4" to match the regular expression '^[a-z][a-zA-Z0-9]*$'.

L7

2min effort

Rename this local variable to match the regular expression '^[a-z][a-zA-Z0-9]*$'.

L9

2min effort

Rename this local variable to match the regular expression '^[a-z][a-zA-Z0-9]*$'.

L9

2min effort

Rename this local variable to match the regular expression '^[a-z][a-zA-Z0-9]*$'.

L9

2min effort

Rename this local variable to match the regular expression '^[a-z][a-zA-Z0-9]*$'.

L9

2min effort

src/custom/fattree/FatTreeFlowClassifier.java

Make flowSizesPerDuration a static final constant or non-public and provide accessors if needed.

L13

10min effort

Make outgoingTraffic a static final constant or non-public and provide accessors if needed.

L14

10min effort

Make flowTable a static final constant or non-public and provide accessors if needed.

L15

10min effort

src/custom/fattree/FatTreeRoutingAlgorithm.java

Add a "clone()" method to this class.

L12

30min effort

Make suffixTables a static final constant or non-public and provide accessors if needed.

L15

10min effort

Replace this use of System.out or System.err by a logger.

L137

10min effort

Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.

L257

5min effort

src/custom/smallworld/GridGraph.java

Replace the synchronized class "Stack" by an unsynchronized one such as "Deque".

L36

20min effort

Do something with the "boolean" value returned by "createNewFile".

L258

15min effort

Use try-with-resources or close this "FileWriter" in a "finally" clause.

L260

5min effort

Do something with the "boolean" value returned by "createNewFile".

L278

15min effort

Use try-with-resources or close this "FileWriter" in a "finally" clause.

L280

5min effort

src/events/DReachingENBEvent.java

Make this "public static type" field final

L20

20min effort

Make type a static final constant or non-public and provide accessors if needed.

L20

10min effort

Make the enclosing method "static" or remove this set.

L67

20min effort

Make the enclosing method "static" or remove this set.

L71

20min effort

src/events/EMovingInSwitchEvent.java

Make type a static final constant or non-public and provide accessors if needed.

L20

10min effort

src/graph/Coordination.java

Remove this unused "nRow" private field.

L6

5min effort

Remove the "nRow" field and declare it as a local variable in the relevant methods.

L6

5min effort

Define and throw a dedicated exception instead of using a generic one.

L24

20min effort

src/graph/Graph.java

Rename this field "V" to match the regular expression '^[a-z][a-zA-Z0-9]*$'.

L7

2min effort

Rename this field "E" to match the regular expression '^[a-z][a-zA-Z0-9]*$'.

L8

2min effort

Rename this method name to match the regular expression '^[a-z][a-zA-Z0-9]*$'.

L36

5min effort

Remove this unused method parameter "path".

L56

5min effort

Remove this method and declare a constant for this value.

L57

5min effort

src/infrastructure/element/Buffer.java

Make physicalLayer a static final constant or non-public and provide accessors if needed.

L9

10min effort

src/infrastructure/element/Element.java

Refactor this method to not always return the same value.

L54

6min effort

Remove this conditional structure or edit its code blocks so that they're not all the same.

L58

15min effort

Replace this if-then-else statement by a single return statement.

L58

2min effort

src/infrastructure/element/LimitedBuffer.java

Define a constant instead of duplicating this literal "ERROR: Buffer: " 3 times.

L34

8min effort

Define a constant instead of duplicating this literal " oversized" 3 times.

L34

8min effort

Replace this use of System.out or System.err by a logger.

L34

10min effort

Replace this use of System.out or System.err by a logger.

L58

10min effort

Replace this use of System.out or System.err by a logger.

L69

10min effort

src/infrastructure/entity/Node.java

Make physicalLayer a static final constant or non-public and provide accessors if needed.

L10

10min effort

Make dataLinkLayer a static final constant or non-public and provide accessors if needed.

L11

10min effort

src/infrastructure/event/Event.java

Rename this class.

L22

5min effort

Rename this constant name to match the regular expression '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'.

L26

2min effort

src/infrastructure/state/State.java

Rename this constant name to match the regular expression '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'.

L19

2min effort

Rename this constant name to match the regular expression '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'.

L20

2min effort

Rename this constant name to match the regular expression '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'.

L22

2min effort

Make element a static final constant or non-public and provide accessors if needed.

L23

10min effort

Make type a static final constant or non-public and provide accessors if needed.

L24

10min effort

Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.

L111

5min effort

src/javatuples/Decade.java

19 duplicated blocks of code must be removed.

3h20min effort

Make "val0" transient or serializable.

L34

30min effort

Make "val1" transient or serializable.

L35

30min effort

Make "val2" transient or serializable.

L36

30min effort

Make "val3" transient or serializable.

L37

30min effort

Make "val4" transient or serializable.

L38

30min effort

Make "val5" transient or serializable.

L39

30min effort

Make "val6" transient or serializable.

L40

30min effort

Make "val7" transient or serializable.

L41

30min effort

Make "val8" transient or serializable.

L42

30min effort

Make "val9" transient or serializable.

L43

30min effort

Method has 10 parameters, which is greater than 7 authorized.

L45

20min effort

Refactor this method to reduce its Cognitive Complexity from 29 to the 15 allowed.

L115

19min effort

Constructor has 10 parameters, which is greater than 7 authorized.

L221

20min effort

src/javatuples/Ennead.java

19 duplicated blocks of code must be removed.

3h20min effort

Make "val0" transient or serializable.

L33

30min effort

Make "val1" transient or serializable.

L34

30min effort

Make "val2" transient or serializable.

L35

30min effort

Make "val3" transient or serializable.

L36

30min effort

Make "val4" transient or serializable.

L37

30min effort

Make "val5" transient or serializable.

L38

30min effort

Make "val6" transient or serializable.

L39

30min effort

Make "val7" transient or serializable.

L40

30min effort

Make "val8" transient or serializable.

L41

30min effort

Method has 9 parameters, which is greater than 7 authorized.

L43

20min effort

Refactor this method to reduce its Cognitive Complexity from 27 to the 15 allowed.

L113

17min effort

Constructor has 9 parameters, which is greater than 7 authorized.

L212

20min effort

src/javatuples/KeyValue.java

3 duplicated blocks of code must be removed.

40min effort

Make "key" transient or serializable.

L26

30min effort

Make "value" transient or serializable.

L27

30min effort

src/javatuples/LabelValue.java

3 duplicated blocks of code must be removed.

40min effort

Make "label" transient or serializable.

L26

30min effort

Make "value" transient or serializable.

L27

30min effort

src/javatuples/Octet.java

15 duplicated blocks of code must be removed.

2h40min effort

Make "val0" transient or serializable.

L32

30min effort

Make "val1" transient or serializable.

L33

30min effort

Make "val2" transient or serializable.

L34

30min effort

Make "val3" transient or serializable.

L35

30min effort

Make "val4" transient or serializable.

L36

30min effort

Make "val5" transient or serializable.

L37

30min effort

Make "val6" transient or serializable.

L38

30min effort

Make "val7" transient or serializable.

L39

30min effort

Method has 8 parameters, which is greater than 7 authorized.

L41

20min effort

Refactor this method to reduce its Cognitive Complexity from 25 to the 15 allowed.

L109

15min effort

Constructor has 8 parameters, which is greater than 7 authorized.

L201

20min effort

src/javatuples/Pair.java

3 duplicated blocks of code must be removed.

40min effort

Make "val0" transient or serializable.

L25

30min effort

Make "val1" transient or serializable.

L26

30min effort

Method has 8 parameters, which is greater than 7 authorized.

L263

20min effort

Method has 8 parameters, which is greater than 7 authorized.

L270

20min effort

Method has 8 parameters, which is greater than 7 authorized.

L277

20min effort

Method has 8 parameters, which is greater than 7 authorized.

L460

20min effort

src/javatuples/Quartet.java

4 duplicated blocks of code must be removed.

50min effort

Make "val0" transient or serializable.

L27

30min effort

Make "val1" transient or serializable.

L28

30min effort

Make "val2" transient or serializable.

L29

30min effort

Make "val3" transient or serializable.

L30

30min effort

Refactor this method to reduce its Cognitive Complexity from 17 to the 15 allowed.

L99

7min effort

src/javatuples/Quintet.java

6 duplicated blocks of code must be removed.

1h10min effort

Make "val0" transient or serializable.

L29

30min effort

Make "val1" transient or serializable.

L30

30min effort

Make "val2" transient or serializable.

L31

30min effort

Make "val3" transient or serializable.

L32

30min effort

Make "val4" transient or serializable.

L33

30min effort

Refactor this method to reduce its Cognitive Complexity from 19 to the 15 allowed.

L102

9min effort

src/javatuples/Septet.java

11 duplicated blocks of code must be removed.

2h effort

Make "val0" transient or serializable.

L31

30min effort

Make "val1" transient or serializable.

L32

30min effort

Make "val2" transient or serializable.

L33

30min effort

Make "val3" transient or serializable.

L34

30min effort

Make "val4" transient or serializable.

L35

30min effort

Make "val5" transient or serializable.

L36

30min effort

Make "val6" transient or serializable.

L37

30min effort

Refactor this method to reduce its Cognitive Complexity from 23 to the 15 allowed.

L106

13min effort

src/javatuples/Sextet.java

8 duplicated blocks of code must be removed.

1h30min effort

Make "val0" transient or serializable.

L30

30min effort

Make "val1" transient or serializable.

L31

30min effort

Make "val2" transient or serializable.

L32

30min effort

Make "val3" transient or serializable.

L33

30min effort

Make "val4" transient or serializable.

L34

30min effort

Make "val5" transient or serializable.

L35

30min effort

Refactor this method to reduce its Cognitive Complexity from 21 to the 15 allowed.

L104

11min effort

src/javatuples/Triplet.java

3 duplicated blocks of code must be removed.

40min effort

Make "val0" transient or serializable.

L26

30min effort

Make "val1" transient or serializable.

L27

30min effort

Make "val2" transient or serializable.

L28

30min effort

src/javatuples/Tuple.java

Make "valueArray" transient or serializable.

L25

30min effort

Make "valueList" transient or serializable.

L26

30min effort

Do not forget to remove this deprecated code someday.

L34

10min effort

src/javatuples/Unit.java

1 duplicated blocks of code must be removed.

20min effort

Make "val0" transient or serializable.

L24

30min effort

Method has 8 parameters, which is greater than 7 authorized.

L219

20min effort

Method has 8 parameters, which is greater than 7 authorized.

L226

20min effort

Method has 9 parameters, which is greater than 7 authorized.

L233

20min effort

Method has 9 parameters, which is greater than 7 authorized.

L240

20min effort

Method has 8 parameters, which is greater than 7 authorized.

L396

20min effort

Method has 9 parameters, which is greater than 7 authorized.

L407

20min effort

src/network/Topology.java

Make cordOfNodes a static final constant or non-public and provide accessors if needed.

L26

10min effort

Make pairGenerator a static final constant or non-public and provide accessors if needed.

L29

10min effort

Remove this useless assignment to local variable "sourceNodeIDs".

L144

15min effort

Remove this useless assignment to local variable "destinationNodeIDs".

L145

15min effort

src/network/elements/EntranceBuffer.java

Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.

L19

5min effort

src/network/elements/ExitBuffer.java

Replace this use of System.out or System.err by a logger.

L38

10min effort

Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.

L46

5min effort

src/network/elements/Packet.java

Rename this constant name to match the regular expression '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'.

L15

2min effort

Make dataLinkLayer a static final constant or non-public and provide accessors if needed.

L17

10min effort

src/network/entities/Host.java

Make type a static final constant or non-public and provide accessors if needed.

L12

10min effort

Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.

L19

5min effort

Change this instance-reference to a static reference.

L63

5min effort

src/network/entities/Link.java

Use static access with "weightedloadexperiment.pairstrategies.PairGenerator" for "isOversubscriptedLink".

L55

5min effort

src/network/entities/Switch.java

Rename this constant name to match the regular expression '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'.

L6

2min effort

src/network/entities/TypeOfHost.java

Rename this constant name to match the regular expression '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'.

L4

2min effort

Rename this constant name to match the regular expression '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'.

L4

2min effort

Rename this constant name to match the regular expression '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'.

L4

2min effort

src/network/layers/DataLinkLayer.java

Make packet a static final constant or non-public and provide accessors if needed.

L6

10min effort

Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.

L13

5min effort

src/network/layers/Layer.java

Make routingAlgorithm a static final constant or non-public and provide accessors if needed.

L7

10min effort

src/network/layers/NetworkLayer.java

Replace this use of System.out or System.err by a logger.

L63

10min effort

src/network/layers/PhysicalLayer.java

Make exitBuffers a static final constant or non-public and provide accessors if needed.

L12

10min effort

The type of the "exitBuffers" object should be an interface such as "Map" rather than the implementation "HashMap".

L12

10min effort

Make entranceBuffers a static final constant or non-public and provide accessors if needed.

L13

10min effort

The type of the "entranceBuffers" object should be an interface such as "Map" rather than the implementation "HashMap".

L13

10min effort

Make sourceQueue a static final constant or non-public and provide accessors if needed.

L14

10min effort

Make links a static final constant or non-public and provide accessors if needed.

L15

10min effort

The type of the "links" object should be an interface such as "Map" rather than the implementation "HashMap".

L15

10min effort

Make simulator a static final constant or non-public and provide accessors if needed.

L16

10min effort

Make node a static final constant or non-public and provide accessors if needed.

L17

10min effort

Remove this unused method parameter "k".

L34

5min effort

src/network/states/enb/N0.java

Complete the task associated to this TODO comment.

L31

Merge this if statement with the enclosing one.

L32

5min effort

src/network/states/enb/N1.java

Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.

L13

5min effort

src/network/states/packet/StateP1.java

Make packet a static final constant or non-public and provide accessors if needed.

L10

10min effort

Remove this unused method parameter "ev".

L12

5min effort

Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.

L18

5min effort

src/network/states/packet/StateP2.java

Make packet a static final constant or non-public and provide accessors if needed.

L10

10min effort

Remove this unused method parameter "ev".

L12

5min effort

Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.

L18

5min effort

src/network/states/packet/StateP3.java

Make packet a static final constant or non-public and provide accessors if needed.

L11

10min effort

Remove this unused method parameter "ev".

L13

5min effort

Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.

L19

5min effort

src/network/states/packet/StateP4.java

Make packet a static final constant or non-public and provide accessors if needed.

L11

10min effort

Remove this unused method parameter "ev".

L13

5min effort

Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.

L19

5min effort

src/network/states/packet/StateP5.java

Make packet a static final constant or non-public and provide accessors if needed.

L11

10min effort

Remove this unused method parameter "ev".

L13

5min effort

Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.

L19

5min effort

src/network/states/packet/StateP6.java

Make packet a static final constant or non-public and provide accessors if needed.

L10

10min effort

Remove this unused method parameter "ev".

L12

5min effort

Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.

L18

5min effort

src/network/states/sourcequeue/Sq2.java

Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.

L13

5min effort

src/.../states/unidirectionalway/W1.java

Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.

L14

5min effort

src/.../states/unidirectionalway/W2.java

Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.

L14

5min effort

src/simulator/DiscreteEventSimulator.java

Make numReceived a static final constant or non-public and provide accessors if needed.

L10

10min effort

Make receivedPacketPerUnit a static final constant or non-public and provide accessors if needed.

L11

10min effort

Move the array designator from the variable to the type.

L11

5min effort

Make numSent a static final constant or non-public and provide accessors if needed.

L12

10min effort

Make numLoss a static final constant or non-public and provide accessors if needed.

L13

10min effort

Make totalPacketTime a static final constant or non-public and provide accessors if needed.

L14

10min effort

Make numEvent a static final constant or non-public and provide accessors if needed.

L15

10min effort

Make totalHop a static final constant or non-public and provide accessors if needed.

L20

10min effort

Make sizeOfCurrEvents a static final constant or non-public and provide accessors if needed.

L22

10min effort

Make halfSizeOfEvents a static final constant or non-public and provide accessors if needed.

L26

10min effort

Rename this field "IS_LIMIT" to match the regular expression '^[a-z][a-zA-Z0-9]*$'.

L28

2min effort

Rename field "VERBOSE" to prevent any misunderstanding/clash with field "verbose" defined on line 18.

L29

10min effort

Rename this field "VERBOSE" to match the regular expression '^[a-z][a-zA-Z0-9]*$'.

L29

2min effort

Rename this field "TIME_LIMIT" to match the regular expression '^[a-z][a-zA-Z0-9]*$'.

L30

2min effort

Rename this method name to match the regular expression '^[a-z][a-zA-Z0-9]*$'.

L32

5min effort

Replace this use of System.out or System.err by a logger.

L87

10min effort

Introduce a new variable instead of reusing the parameter "ev".

L92

5min effort

Remove this unused method parameter "currentTime".

L135

5min effort

src/simulator/Simulator.java

Rename this class.

L9

5min effort

Make topology a static final constant or non-public and provide accessors if needed.

L14

10min effort

Make currentEvents a static final constant or non-public and provide accessors if needed.

L16

10min effort

src/weightedloadexperiment/GraphPanel.java

Remove these useless parentheses.

L86

1min effort

Save and re-use this "Random".

L187

5min effort

Use static access with "javax.swing.WindowConstants" for "EXIT_ON_CLOSE".

L196

5min effort

Make this anonymous inner class a lambda (sonar.java.source not set. Assuming 8 or greater.)

L204

5min effort

Use static access with "javax.swing.WindowConstants" for "EXIT_ON_CLOSE".

L216

5min effort

src/weightedloadexperiment/ThroughputExperiment.java

Replace this use of System.out or System.err by a logger.

L37

10min effort

Remove this unused "count" local variable.

L42

5min effort

Iterate over the "entrySet" instead of the "keySet".

L43

5min effort

Remove this useless assignment to local variable "rawThroughput".

L77

15min effort

Remove this unused "rawThroughput" local variable.

L77

5min effort

Cast one of the operands of this division operation to a "double".

L79

5min effort

Remove this useless assignment to local variable "alternativeRawThroughput".

L80

15min effort

Remove this unnecessary cast to "double".

L80

5min effort

Replace this use of System.out or System.err by a logger.

L84

10min effort

Remove this unused "rxPacket" local variable.

L114

5min effort

Remove this unused "thp" local variable.

L115

5min effort

Replace this use of System.out or System.err by a logger.

L133

10min effort

Iterate over the "entrySet" instead of the "keySet".

L139

5min effort

Replace this use of System.out or System.err by a logger.

L140

10min effort

This block of commented-out lines of code should be removed.

L148

5min effort

This block of commented-out lines of code should be removed.

L150

5min effort

This block of commented-out lines of code should be removed.

L156

5min effort

This block of commented-out lines of code should be removed.

L162

5min effort

src/.../pairstrategies/ForcePair.java

2 duplicated blocks of code must be removed.

30min effort

"modulo" is the name of a field in "OverSubscription".

L10

5min effort

Replace this use of System.out or System.err by a logger.

L30

10min effort

src/.../pairstrategies/InterPodIncoming.java

1 duplicated blocks of code must be removed.

20min effort

Remove the "adjMx" field and declare it as a local variable in the relevant methods.

L16

5min effort

Remove this expression which always evaluates to "true"

L88

10min effort

Remove this expression which always evaluates to "true"

L97

10min effort

Method has 8 parameters, which is greater than 7 authorized.

L104

20min effort

Introduce a new variable instead of reusing the parameter "expectedSrc".

L107

5min effort

Remove this useless assignment to local variable "found".

L109

15min effort

Introduce a new variable instead of reusing the parameter "found".

L109

5min effort

Remove this useless assignment to local variable "currPod".

L113

15min effort

Introduce a new variable instead of reusing the parameter "currPod".

L113

5min effort

Remove this useless assignment to local variable "currPod".

L115

15min effort

Introduce a new variable instead of reusing the parameter "currPod".

L115

5min effort

Remove this useless assignment to local variable "found".

L126

15min effort

Introduce a new variable instead of reusing the parameter "found".

L126

5min effort

Remove this useless assignment to local variable "currPod".

L134

15min effort

Introduce a new variable instead of reusing the parameter "currPod".

L134

5min effort

Remove this useless assignment to local variable "currPod".

L136

15min effort

Introduce a new variable instead of reusing the parameter "currPod".

L136

5min effort

Remove this unused "equal" local variable.

L154

5min effort

Iterate over the "entrySet" instead of the "keySet".

L155

5min effort

Replace this use of System.out or System.err by a logger.

L173

10min effort

Introduce a new variable instead of reusing the parameter "realCore".

L176

5min effort

Replace this use of System.out or System.err by a logger.

L177

10min effort

Introduce a new variable instead of reusing the parameter "realCore".

L198

5min effort

Replace this use of System.out or System.err by a logger.

L199

10min effort

Replace this use of System.out or System.err by a logger.

L209

10min effort

src/.../pairstrategies/OverSubscription.java

1 duplicated blocks of code must be removed.

20min effort

Make modulo a static final constant or non-public and provide accessors if needed.

L18

10min effort

Make k a static final constant or non-public and provide accessors if needed.

L20

10min effort

Make routing a static final constant or non-public and provide accessors if needed.

L22

10min effort

Make graph a static final constant or non-public and provide accessors if needed.

L23

10min effort

Rename "k" which hides the field declared at line 20.

L40

5min effort

Remove this unused method parameter "destination".

L62

5min effort

src/.../pairstrategies/SameIDOutgoing.java

Refactor this method to reduce its Cognitive Complexity from 20 to the 15 allowed.

L55

10min effort

Reduce the total number of break and continue statements in this loop to use at most one.

L61

20min effort

Remove this useless assignment to local variable "i".

L97

15min effort

Replace this use of System.out or System.err by a logger.

L98

10min effort

Replace this use of System.out or System.err by a logger.

L103

10min effort

Replace this use of System.out or System.err by a logger.

L106

10min effort

Replace this use of System.out or System.err by a logger.

L107

10min effort

Replace this use of System.out or System.err by a logger.

L109

10min effort

Remove this unused private "isSameSubNet" method.

L126

5min effort

Replace this if-then-else statement by a single return statement.

L130

2min effort

Replace this use of System.out or System.err by a logger.

L183

10min effort

Replace this use of System.out or System.err by a logger.

L186

10min effort

Replace this use of System.out or System.err by a logger.

L195

10min effort

src/.../pairstrategies/interpod/MinimalCoreSwitches.java

Remove the "coreInPath" field and declare it as a local variable in the relevant methods.

L12

5min effort

Remove this useless assignment to local variable "usedPods".

L56

15min effort

Remove this expression which always evaluates to "true"

L95

10min effort

Remove this expression which always evaluates to "true"

L109

10min effort

Replace this if-then-else statement by a single return statement.

L129

2min effort

Filters